-
Notifications
You must be signed in to change notification settings - Fork 6
Eager workflow start sample #55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
cretz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor things
| * [context_propagation](context_propagation) - Use interceptors to propagate thread/fiber local data from clients | ||
| through workflows/activities. | ||
| * [dsl](dsl) - Demonstrates having a workflow interpret/invoke arbitrary steps defined in a DSL. | ||
| * [eager_wf_start](eager_wf_start) - Demonstrates Eager Workflow Start to reduce latency for workflows that start with a local activity. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similar to .NET suggestion, can we use the full term workflow here?
| require_relative 'eager_workflow' | ||
| require_relative 'greeting_activity' | ||
|
|
||
| TASK_QUEUE = 'eager-wf-start-task-queue' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Usually our samples have a <sample-name>-sample task queue name, so can we remove -task-queue and add -sample?
| handle = client.start_workflow( | ||
| EagerWfStart::EagerWorkflow, | ||
| 'Temporal', | ||
| id: "eager-workflow-id-#{SecureRandom.uuid}", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Our samples have traditionally not used random identifiers
What was changed
Simple sample showing how to use eager workflow start with a local activity.
Why?
Show a sample use case of eager workflow start (i.e. a workflow with a local activity).
How was this tested:
Sample runs correctly. There does not seem to be a way to verify that a worker has started eagerly in ruby SDK
Any docs updates needed?
No